Repository navigation
docs(reporting): update shared row storage migration guidance - #1879
Conversation
![aao-secretariat[bot]](https://avatars.githubusercontent.com/u/221690821?s=60&v=4){kind=small}
There was a problem hiding this comment.
Ladon verdict: Approve
Approve — docs-only change, no blocking findings.
This PR touches a single non-generated documentation file (docs/reporting-row-storage.md, +10/−9). It reframes the shared-row-storage FK guidance to match the SDK 10 / reporting-storage 0.2 line (no fixed Python/JS release creates the obligation_id FK; legacy DBs apply JS migration 0011 or drop the constraint) and quotes the constraint identifier in both SQL blocks to match the surrounding prose. No wire-shape, public-surface, or semver impact; coherent with untouched layout-notes text.
No gated paths (gated_paths: false), no high-risk flag, no no-auto-approve team match, no prior decision. Reviewer surfaced zero Critical/High/Medium findings. None of decision rows 1–8 fire → row 9 approve.
Carry the shared row-storage migration guidance to the SDK 10 / reporting-storage 0.2 release line. The readiness refusal already supplies an executable schema-qualified statement with the actual quoted constraint name, including embedded quotes.
Tested: 28 row-storage PostgreSQL tests and 4 cross-SDK interoperability tests.
Related: #1495, adcontextprotocol/adcp-client#3651 (JavaScript fix), #1880 (main release line).